-
Notifications
You must be signed in to change notification settings - Fork 786
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Backport powhegUL to master branch #3549
Conversation
Hi @Cvico, thanks a lot for this! Can I ask: did you try to compile Powheg in the master branch already? I remember that I had trouble with that on master, but it might be fixed by now. Cheers, |
Hi Laurids, I'll have a look into PDFs then. I agree that it's better to reduce. About compiling in master, I've been doing tests and I was able to include it within the master branch and compile, but only with CMSSW_10_6_X releases. I think the main problem is with the fortran version in CMSSW_12_X, but I think powheg gridpacks are agnostic to what release they were made in. Maybe @agrohsje can comment on this, I took this info from this thread in CMSTalk: https://cms-talk.web.cern.ch/t/hzz-failure-compiling-fortran-for-hzj-powheg-process/21259/4. So other than checking that this compiles in 10_6_X, I've not done anything more. |
Indeed, for the time being, we can run in the sl7 10_6 environment on el8 nodes. Our new Powheg contact is looking into fixing these problems. But he just started, so it might take some time. |
Hi, I've updated the PR with the PDF configs from Run2 UL gridpacks |
Can we merge this PR, now? |
This is quite old but I think we can merge, right? We should resolve the conflict in |
Dear @bbilin @lviliani @Cvico @lauridsj @sihyunjeon @DickyChant . This PR is a core PR to run bb4l in cms. It is more than 1 year old. If we don't merge such PRs in a timely manner we are screwed. Who is now in charge to judge this PR and fix it? |
Hi, from what I can see in the code it looks fine. I never tested it in the master branch, but I think @Cvico did, so it should be fine to merge. If you want to be sure, someone could test compiling bb4l with this branch. |
Hi, I did get it to work 1 year ago. Just for sanity check, I can try to see if it still works in EL8 (this was done back when we had CC7 on lxplus). If that works, I would just remake the PR... Although I don't have an strong opinion there. Which CMSSW version should I use? I understand current production is based on el8, so a 13_3_X should do the trick. But I'm not sure if my understanding is correct. This is relevant in powheg processes because fortran compilation is often quite annoying (I did a small try on 13_3_X right now and I get a fortran compilation error in openloops because types cannot be initialized in the same way for newer fotran versions). This can be fixed but it's better if we can settle on which version. |
I can redo the PR once this #3716 is accepted. I don't mind redoing, and we can make sure time it gets merged promptly. By the way, just to mention that (for this particular PR ) I'm of course also partly to blame of course, I made the PR and forgot about the status because it got obscured by many other things. Very busy schedules in GEN related stuff! For sake of proactivity let's put things in order and I can remake the PR. This time I'll put extra effort on making sure we get it merged in a timely manner as @agrohsje suggests :D |
As it was written by Carlos, this summer it was busy with many other activities, including the production of this very bb4l samples.. So in the end we were using the setup without getting it merged into the genproductions. In general we need to define a better review policy for MG, Powheg, Sherpa or other generators, involving the generator contacts in a more active way and NOT waiting for two L2 conveners following EVERYTHING. |
Dear all,
This PR aims to introduce the b_bbar_4l process (work done by @lauridsj and @agrohsje) in genproductions so it can
be used for Run3 bb4l production. I took cards and required patches from Laurids' PR: pull 3493.
One thing I did not modify is the PDF sets in the
make_rwl.py
script. So in principle this is using the defaultPDF sets from Run3. I'm not sure if we want to reduce the amount of sets (to reduce computational costs) for
this particular case, since b_bb4l reweighting can be quite consuming.
I think other required patches are already merged in genproductions. In particular I'm refering to pull 3477, which seems to be already merged in master.
Thanks in advance,
Carlos Vico